xl, e820_host, PV passthrough: Fix guests crashing when memory == maxmem
authorKonrad Rzeszutek Wilk <konrad@kernel.org>
Fri, 24 May 2013 13:36:16 +0000 (09:36 -0400)
committerIan Campbell <ian.campbell@citrix.com>
Thu, 30 May 2013 08:01:24 +0000 (09:01 +0100)
commitd21b46e9b723b860ab6e32f6ad96dd4c650596c7
treefb8186530e76a9b39ec653943a043c370d126c12
parent41abbadef60e5fccdfd688579dd458f7f7887cf5
xl, e820_host, PV passthrough: Fix guests crashing when memory == maxmem

The code had an obvious bug where it would assume that the balloon
amount would always be _something_ and add an E820_RAM entry at the
end of the E820 array. The added E820_RAM would contain the balloon amount
plus the delta of memory that had to be subtracted b/c of the various
E820 entries. That assumption is certainly true when maxmem != mem,
but if guest config has maxmem = memory that is incorrect (as balloon
value is zero). The end result is that the E820 that is constructed
is missing a swath of "delta" memory and in most cases ends up with
only one E820_RAM entry that is of 512MB size on many Intel systems.

Reported-by: Christian Holpert <christian@holpert.de>
Signed-off-by: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
Acked-by: Ian Campbell <ian.campbell@citrix.com>
tools/libxl/libxl_x86.c